Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix info.json keys in approved CC guide #6382

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

Flame442
Copy link
Member

Description of the changes

Like many people, I simply referenced other people's cogs to make my info.jsons, and an extra phantom key from the V2 days made its way into our docs. I removed the name key from both the repo and cog requirements, and added the conditional min_python_version key since lots of cogs lately have needed that one.

Technically the cookie cutter repo and the policies should also be updated, but I don't want to do that :(

Have the changes in this PR been tested?

Yes

@Flame442 Flame442 added the Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing. label May 23, 2024
@Flame442 Flame442 added this to the 3.5.10 milestone May 23, 2024
@github-actions github-actions bot added Category: Docs - For Developers This is related to developer (cog creator) documentation. Docs-only labels May 23, 2024
@Jackenmen
Copy link
Member

Technically the cookie cutter repo and the policies should also be updated, but I don't want to do that :(

Arguable - it's still an existing convention to put cog's class name in the name key, even if it doesn't currently do anything. I agree that it doesn't really make sense to have it listed in the approved CC guide though.

@Jackenmen Jackenmen modified the milestones: 3.5.11, 3.5.10 Jul 10, 2024
@Jackenmen Jackenmen merged commit 57b76bc into Cog-Creators:V3/develop Jul 10, 2024
18 checks passed
@red-githubbot red-githubbot bot added the Changelog Entry: Pending Changelog entry for this PR hasn't been added by repo maintainers yet. label Jul 10, 2024
@Jackenmen Jackenmen added Changelog Entry: Added Changelog entry for this PR has already been added to changelog PR. and removed Changelog Entry: Pending Changelog entry for this PR hasn't been added by repo maintainers yet. labels Jul 10, 2024
BenCos17 added a commit to JARVIS-discordbot/Red-DiscordBot-jarvis that referenced this pull request Jul 11, 2024
thanks Redbot team

* Docs: remove Atom from the list of recommended editors (Cog-Creators#6388)

* Fix info.json keys in approved CC guide (Cog-Creators#6382)

* Use YouTube source plugin over the deprecated built-in source (Cog-Creators#6373)

Co-authored-by: aikaterna <20862007+aikaterna@users.noreply.github.com>

* Bump d.py version to 2.4.0 (Cog-Creators#6401)

Co-authored-by: Ryan <yamikaitou@gmail.com>

* Fix `@commands.can_manage_channel` always passing (Cog-Creators#6398)

* Bump dependencies (Cog-Creators#6402)

* Red 3.5.10 - Changelog (Cog-Creators#6403)

* Automated Crowdin downstream (Cog-Creators#6405)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Version bump to 3.5.10 (Cog-Creators#6404)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Last-minute changelog fixes

* Version bump to 3.5.11.dev1 (Cog-Creators#6406)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: Seaswimmer <102361830+SeaswimmerTheFsh@users.noreply.github.com>
Co-authored-by: Michael Oliveira <34169552+Flame442@users.noreply.github.com>
Co-authored-by: Jakub Kuczys <me@jacken.men>
Co-authored-by: aikaterna <20862007+aikaterna@users.noreply.github.com>
Co-authored-by: Ryan <yamikaitou@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Dav-Git pushed a commit to Dav-Git/Red-DiscordBot that referenced this pull request Sep 8, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Category: Docs - For Developers This is related to developer (cog creator) documentation. Changelog Entry: Added Changelog entry for this PR has already been added to changelog PR. Docs-only Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants